Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add SSH key for all relevant workflows #97

Merged
merged 2 commits into from
Oct 4, 2024
Merged

Add SSH key for all relevant workflows #97

merged 2 commits into from
Oct 4, 2024

Conversation

penelopeysm
Copy link
Member

@penelopeysm penelopeysm commented Sep 26, 2024

This PR adds SSH keys for CompatHelper, Documenter, and TagBot workflows, so that they actually work properly.

(Context: TuringLang/Turing.jl#2347 requires docs for AdvancedVI 0.2, and the lack of a SSH key here means that Documenter currently only generates 'dev' docs i.e. current master branch. ... Although I notice that AdvancedVI 0.2 has no docs, so this PR alone doesn't fully solve the problem. Once this is merged I'll follow up with a PR to the v0.2-backport branch.)

By the way, the buildkite integration hasn't ever passed on the master branch. I propose to also remove it.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Benchmark Results

Benchmark suite Current: 6ddc37f Previous: 95a8b55 Ratio
normal + bijector/meanfield/ForwardDiff 488783352 ns
normal + bijector/meanfield/ReverseDiff 193833703 ns

This comment was automatically generated by workflow using github-action-benchmark.

@Red-Portal
Copy link
Member

Red-Portal commented Sep 27, 2024

Hi @penelopeysm , the buildkite integration was failing because it is currently not configured. I just added an empty pipeline in #98 so that should fix that problem.

@Red-Portal
Copy link
Member

Is this ready to merge?

@penelopeysm
Copy link
Member Author

Sorry I forgot about this! Yes, it should be, at long as you are happy. I'll fix merge conflicts.

@Red-Portal
Copy link
Member

Looks good to me. Thanks!

@Red-Portal Red-Portal merged commit 4da2f5b into master Oct 4, 2024
11 checks passed
@Red-Portal Red-Portal deleted the py/gha branch October 4, 2024 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants